Skip to content

Add clang-format and clang-tidy targets and github workflows#111

Closed
fbaeuerle wants to merge 5 commits intoeclipse-score:mainfrom
fbaeuerle:fb/bazel-tools-cc
Closed

Add clang-format and clang-tidy targets and github workflows#111
fbaeuerle wants to merge 5 commits intoeclipse-score:mainfrom
fbaeuerle:fb/bazel-tools-cc

Conversation

@fbaeuerle
Copy link
Contributor

No description provided.

bazel-tools-cc and bazel-tools-python currently still require python
3.8, so in order to allow using the latest release of those two bazel
modules, add a version override to bring back support for python 3.8
which rules_python dropped with version 1.8.0.
However, the checks are currently failing due to the strict need of
allowing reversible transformations back to the bmw-internal repository.
@fbaeuerle fbaeuerle temporarily deployed to workflow-approval March 11, 2026 12:40 — with GitHub Actions Inactive
@fbaeuerle fbaeuerle temporarily deployed to workflow-approval March 11, 2026 12:40 — with GitHub Actions Inactive
@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

And let it pass on failure, for now.
Which currently runs clang-tidy. Scores' cicd-workflows also contains a
static-analysis workflow, this one, however, seems to be tailored
towards rust.
@fbaeuerle fbaeuerle temporarily deployed to workflow-approval March 11, 2026 12:47 — with GitHub Actions Inactive
@fbaeuerle fbaeuerle changed the title Add clang-format and clang-tidy targets and github workflos Add clang-format and clang-tidy targets and github workflows Mar 11, 2026
@4og
Copy link
Member

4og commented Mar 16, 2026

The workflows are merged to main. I'm closing this.

@4og 4og closed this Mar 16, 2026
@github-project-automation github-project-automation bot moved this from In Progress to Done in BAS - Baselibs FT Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants